Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Story/1024/add new transformations #1068

Closed
wants to merge 162 commits into from

Conversation

basiliskus
Copy link
Contributor

Add a PR title

Describe what changed in this PR at a high level.

Issue

Add a link to the issue here. Consider using
closing keywords
if the this PR isn't for a story (stories will be closed through different means).

Checklist

  • I have added tests to cover my changes
  • I have added logging where useful (with appropriate log level)
  • I have added JavaDocs where required
  • I have updated the documentation accordingly

Note: You may remove items that are not applicable

basiliskus and others added 30 commits April 23, 2024 16:10
Added dummy definitions from design discussion
Updated one of the failing tests
Use double checked locking on ensureRulesloaded
luis-pabon-tf and others added 28 commits May 3, 2024 11:35
…hub.com:CDCgov/trusted-intermediary into story/1024/transformation-engine-implementation
…n to validate expected changes that will be used in other tests
…hub.com:CDCgov/trusted-intermediary into story/1024/transformation-engine-implementation
Update format to match ValidationRuleEngine
…nterfere with each other. Can make more generic
Adding spot for adding test for new rule
@basiliskus basiliskus closed this May 7, 2024
@basiliskus basiliskus deleted the story/1024/add-new-transformations branch May 7, 2024 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants